Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Тотальный реворк языков #165

Merged
merged 17 commits into from
Aug 6, 2024
Merged

Тотальный реворк языков #165

merged 17 commits into from
Aug 6, 2024

Conversation

FaDeOkno
Copy link
Collaborator

@FaDeOkno FaDeOkno commented Aug 2, 2024 •
edited
Loading

Описание PR

Полный реворк системы языков. Улучшение, оптимизация и доведение до более-менее презентабельного вида.

Медиа

image
image

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

  • tweak: Полный реворк системы языков.
  • tweak: Языки теперь выделяются не названием, а цветом текста.
  • tweak: Упрощена система языков.
  • tweak: Языковые импланты теперь можно извлечь.
  • add: Меню языков теперь открывается при нажатии на кнопку L и имеет кнопку в верхнем меню.

@github-actions github-actions bot added Changes: UI Изменение интерфейса Changes: Sprites Изменение спрайтов Changes: Localization Изменение локализации labels Aug 2, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 2, 2024 •
edited
Loading

RSI Diff Bot; head commit b75b466 merging into e60d69e
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:
Edit: diff updated after b75b466

@FaDeOkno
Copy link
Collaborator Author

FaDeOkno commented Aug 2, 2024

Починю позже

Schrodinger71
Schrodinger71 reviewed Aug 2, 2024
View reviewed changes
Content.Client/Corvax/TTS/TTSSystem.cs Show resolved Hide resolved
Content.Client/Input/ContentContexts.cs Outdated Show resolved Hide resolved
Content.Client/UserInterface/Systems/MenuBar/Widgets/GameTopMenuBar.xaml Show resolved Hide resolved
Content.Server/ADT/Language/LanguageSystem.cs Outdated Show resolved Hide resolved
Content.Server/ADT/Language/TranslatorImplantSystem.cs Show resolved Hide resolved
Content.Server/Corvax/TTS/TTSSystem.cs Outdated Show resolved Hide resolved
Content.Server/Corvax/TTS/TTSSystem.cs Outdated Show resolved Hide resolved
Content.Server/Radio/EntitySystems/HeadsetSystem.cs Outdated Show resolved Hide resolved
Content.Server/Radio/EntitySystems/RadioDeviceSystem.cs Outdated Show resolved Hide resolved
Content.Server/Radio/EntitySystems/RadioSystem.cs Outdated Show resolved Hide resolved
@Schrodinger71 Schrodinger71 requested a review from 1Stepka1 August 2, 2024 08:28
1Stepka1
1Stepka1 requested changes Aug 2, 2024
View reviewed changes
Copy link
Collaborator

@1Stepka1 1Stepka1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так же видно что трогаются части визардов,где-то подписано где-то нет,и,а где-то вообще по другому подписанно,Шредингер уже указал некоторые часть,постарайся более единообразно указывать комментарий изменения

Resources/Prototypes/ADT/Entities/Objects/Device/Translators/handheld.yml Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/Entities/Objects/Device/Translators/implanters.yml Outdated Show resolved Hide resolved
Resources/Prototypes/ADT/Entities/Objects/Device/Translators/implants.yml Outdated Show resolved Hide resolved
@FaDeOkno
Copy link
Collaborator Author

FaDeOkno commented Aug 2, 2024

Ужасы
Поправил всё, вроде

@1Stepka1
Copy link
Collaborator

1Stepka1 commented Aug 4, 2024 •
edited
Loading

Хотел бы увидеть фото или видео демонстрацию работу портативных переводчиков так как по какой-то причине я не смог с ними нормально поработать

Schrodinger71
Schrodinger71 previously requested changes Aug 4, 2024
View reviewed changes
Resources/Prototypes/Entities/Mobs/Player/observer.yml Show resolved Hide resolved
Resources/keybinds.yml Show resolved Hide resolved
@Schrodinger71
Copy link
Collaborator

В целом неплохо уже

@Schrodinger71 Schrodinger71 added Status: Awaiting Changes Ожидание изменений and removed Status: Needs Review labels Aug 6, 2024
@FaDeOkno FaDeOkno dismissed Schrodinger71’s stale review August 6, 2024 17:27

Исправлено

@FaDeOkno FaDeOkno removed the Status: Awaiting Changes Ожидание изменений label Aug 6, 2024
Schrodinger71
Schrodinger71 approved these changes Aug 6, 2024
View reviewed changes
Copy link
Collaborator

@Schrodinger71 Schrodinger71 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Котя пупс

@Schrodinger71 Schrodinger71 added this pull request to the merge queue Aug 6, 2024
Merged via the queue into AdventureTimeSS14:master with commit 8813c92 Aug 6, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Schrodinger71 Schrodinger71 Schrodinger71 approved these changes

@1Stepka1 1Stepka1 1Stepka1 approved these changes

Assignees

@1Stepka1 1Stepka1

@Schrodinger71 Schrodinger71

@FaDeOkno FaDeOkno

Labels
Changes: Localization Изменение локализации Changes: Sprites Изменение спрайтов Changes: UI Изменение интерфейса
Projects
Task Board (Для переноса сборки)
Status: Выполнено
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@FaDeOkno @1Stepka1 @Schrodinger71